You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The pull request refactors and updates the code related to discharge reasons in various components and interfaces. It uses a constant DISCHARGE_REASONS for the discharge reasons and changes the type of the discharge_reason property to a number or null to align with the backend API. This improves the code consistency, clarity, and maintainability.
Change the type of discharge_reason from string to number or null in various interfaces, components, and states to match the backend API and avoid type errors (link, link, link, link, link, link, link)
Replace hard-coded strings with the DISCHARGE_REASONS constant from src/Common/constants.tsx in various components to improve readability and maintainability and avoid errors due to typos or changes in the discharge reasons (link, link, link, link, link, link, link, link, link, link, link, link, link)
Change the label, value, name, and error of the TextFormField and TextAreaFormField components in the DischargeModal component based on the discharge_reason value using numbers and an object literal to access the corresponding text (link, link, link, link, link, link)
@Ashesh3 We truly appreciate your efforts. Thank you for taking the time to contribute; this is a very valuable contribution to us 🥇. We always welcome your contribution 🙂, so feel free to contribute to anything anytime, and never lose that spirit of innovation 🙌.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Ashesh3
removed
changes required
merge conflict
Backend PR: ohcnetwork/care#1773
WHAT
🤖[deprecated] Generated by Copilot at 5693149
The pull request refactors and updates the code related to discharge reasons in various components and interfaces. It uses a constant
DISCHARGE_REASONSfor the discharge reasons and changes the type of thedischarge_reasonproperty to a number or null to align with the backend API. This improves the code consistency, clarity, and maintainability.Proposed Changes
discharge_reasontoIntegerFieldcare#1739@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers
HOW
🤖[deprecated] Generated by Copilot at 5693149
discharge_reasonfrom string to number or null in various interfaces, components, and states to match the backend API and avoid type errors (link, link, link, link, link, link, link)DISCHARGE_REASONSconstant fromsrc/Common/constants.tsxin various components to improve readability and maintainability and avoid errors due to typos or changes in the discharge reasons (link, link, link, link, link, link, link, link, link, link, link, link, link)TextFormFieldandTextAreaFormFieldcomponents in theDischargeModalcomponent based on thedischarge_reasonvalue using numbers and an object literal to access the corresponding text (link, link, link, link, link, link)